Skip to content

fix html clipboard issue (only copy text/plain)#2579

Merged
sawka merged 1 commit intomainfrom
sawka/fix-html-clipboard
Nov 20, 2025
Merged

fix html clipboard issue (only copy text/plain)#2579
sawka merged 1 commit intomainfrom
sawka/fix-html-clipboard

Conversation

@sawka
Copy link
Member

@sawka sawka commented Nov 20, 2025

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025 •
edited
Loading

Walkthrough

A single configuration option is added to the CodeEditor component's Monaco editor options. The copyWithSyntaxHighlighting setting is set to false, which prevents syntax highlighting from being included when users copy content from the editor. This is a minimal runtime behavior modification with no changes to control flow, error handling, or public API signatures.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify the copyWithSyntaxHighlighting option name matches Monaco editor's API documentation
  • Confirm the value false aligns with the intended behavior for copy operations

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess whether the description relates to the changeset. Add a pull request description explaining the clipboard behavior change and the rationale for disabling syntax highlighting in copied content.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix html clipboard issue (only copy text/plain)' directly relates to the changeset which modifies the CodeEditor's copyWithSyntaxHighlighting option to control clipboard behavior.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sawka/fix-html-clipboard
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6420ae9 and 47a8a7a.

📒 Files selected for processing (1)
  • frontend/app/view/codeeditor/codeeditor.tsx (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Build for TestDriver.ai
  • GitHub Check: Analyze (go)
  • GitHub Check: Analyze (javascript-typescript)
🔇 Additional comments (1)
frontend/app/view/codeeditor/codeeditor.tsx (1)

161-161: No issues found—option is valid and correctly implemented.

The copyWithSyntaxHighlighting option exists in monaco-editor v0.52.0 and is properly exposed via IEditorOptions/IStandaloneEditorConstructionOptions. Setting it to false correctly disables HTML/RTF formatting in clipboard copies, aligning with the PR objective to copy only plain text.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sawka sawka merged commit a28d1a1 into main Nov 20, 2025
6 of 7 checks passed
@sawka sawka deleted the sawka/fix-html-clipboard branch November 20, 2025 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@sawka